-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LCFS-1207: Update fuel type for other_uses #1411
Conversation
9c06e6e
to
732e484
Compare
77f163f
to
2fc7a88
Compare
2fc7a88
to
d13cd66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this ticket it was required to add 3 new columns to the other_use table in the frontend. Was this completed in a separate ticket?
@@ -11,7 +11,7 @@ | |||
|
|||
# revision identifiers, used by Alembic. | |||
revision = "cd8698fe40e6" | |||
down_revision = "9206124a098b" | |||
down_revision = "7ae38a8413ab" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please double check this migration id is in the correct order.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updating this another migration appeared before mine.
The addition of the fields was already merged #1269 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Fuel type for other uses should be the following:
Story